Skip to content

Conversation

@shartte
Copy link
Contributor

@shartte shartte commented Dec 14, 2024

NeoGradle companion PR: neoforged/NeoGradle#257

@neoforged-pr-publishing
Copy link

neoforged-pr-publishing bot commented Dec 14, 2024

  • Publish PR to GitHub Packages

Last commit published: ef39da618c6fa969018d4de2743ffe2e41c9bbef.

PR Publishing

The artifacts published by this PR:

Repository Declaration

In order to use the artifacts published by the PR, add the following repository to your buildscript:

repositories {
    maven {
        name 'Maven for PR #50' // https://github.com/neoforged/NeoFormRuntime/pull/50
        url 'https://prmaven.neoforged.net/NeoFormRuntime/pr50'
        content {
            includeModule('net.neoforged', 'neoform-runtime')
        }
    }
}

@shartte
Copy link
Contributor Author

shartte commented Dec 14, 2024

Ah I forgot about extracting the bundled server jar.

@shartte shartte changed the title Support the split manifest Implement generation of split-source Manifest entries Dec 14, 2024
@neoforged-automation neoforged-automation bot added the needs rebase This Pull Request needs to be rebased before being merged label Jan 2, 2025
@neoforged-automation
Copy link

@shartte, this pull request has conflicts, please resolve them for this PR to move forward.

shartte added a commit that referenced this pull request Jun 27, 2025
…#66)

This is a rebased version of #50 (made as a separate PR because I lack
appropriate permissions to do otherwise); from a quick test, its
behaviour seems to match that of
neoforged/NeoGradle#257, putting the generated
manifest info in the MANIFEST.MF of the client-extra jar.

---------

Co-authored-by: Sebastian Hartte <[email protected]>
@shartte shartte closed this Jul 18, 2025
@shartte shartte deleted the split-manifest branch July 18, 2025 22:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs rebase This Pull Request needs to be rebased before being merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant